🌱 Add e2e test for installing bundle with large (~1MB) resources#2602
Conversation
Adds a new large-crd-operator test bundle containing a ~1MB CRD to verify that the Boxcutter runtime correctly handles large bundle installations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
✅ Deploy Preview for olmv1 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
pedjak
changed the title
There was a problem hiding this comment.
Pull request overview
Adds a new “large-crd-operator” test bundle (with a very large CRD manifest) and wires it into the test catalog plus a new e2e scenario to validate Boxcutter can install bundles containing ~1MB CRDs.
Changes:
- Extend the e2e test catalog to publish
large-crd-operatorfromtest-catalog. - Add a new
large-crd-operatorbundle (CSV + supporting ConfigMap + bundle metadata annotations). - Add an e2e scenario that installs the bundle and asserts the CRD and deployment are installed.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| testdata/images/catalogs/test-catalog/v1/configs/catalog.yaml | Publishes the new large-crd-operator package/channel/bundle in the e2e test catalog. |
| testdata/images/bundles/large-crd-operator/v1.0.0/metadata/annotations.yaml | Adds bundle metadata annotations for the new test bundle. |
| testdata/images/bundles/large-crd-operator/v1.0.0/manifests/script.configmap.yaml | Adds the script ConfigMap used by the bundle’s deployment. |
| testdata/images/bundles/large-crd-operator/v1.0.0/manifests/largecrdoperator.clusterserviceversion.yaml | Defines the operator CSV, including the deployment and owned CRD reference. |
| test/e2e/features/install.feature | Adds a BoxcutterRuntime scenario to install the large-CRD bundle and verify key resources are installed. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2602 +/- ##
=======================================
Coverage 68.86% 68.86%
=======================================
Files 139 139
Lines 9872 9872
=======================================
Hits 6798 6798
Misses 2557 2557
Partials 517 517
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
openshift-ci
bot
added
the
approved
openshift-merge-bot
bot
merged commit 46d66e1
into
operator-framework:main
Adds a new large-crd-operator test bundle containing a ~1MB CRD to verify that the Boxcutter runtime correctly handles large bundle installations.
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com